Skip to content

Conversation

@mageshb
Copy link

@mageshb mageshb commented Apr 20, 2021

Fixes #11

mageshb and others added 6 commits February 8, 2021 20:18
OpenAPI
Info
Contact
License
Server
ServerVariable
PathItem
Operation
RequestBody
MediaType
Encoding
Example
Link
Response
Tag
ExternalDocumentation
Responses
Schema
XML
Security Scheme
OAUTH Flows
OAUTH Flow
@maksbotan maksbotan self-requested a review April 21, 2021 11:14
@maksbotan
Copy link
Collaborator

Hi!

Can you please merge master to your branch, so that there are no conflicts?

@mageshb
Copy link
Author

mageshb commented Apr 22, 2021

Sure, Will send in updated one

@blackheaven
Copy link

I have rebased it in my repo, I have made a pull request on @byteally repository, but I can make one here if needed.

-- ],
-- "type": "object"
-- }

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this empty line is crucial, it separates a new section "manipulation" from the previous one. please bring it back :)

-- "title": "Todo API",
-- "version": "1.0"
-- }

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for the catch!

in pairs (pairsToSeries (opts ^. saoAdditionalPairs) <> ps)
where
proxy = Proxy :: Proxy a
defs = hcpure (Proxy :: Proxy AesonDefaultValue) defaultValue
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why is this different from (aesonDefaults proxy) in sopSwaggerGenericToEncoding?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support for Specification Extensions in OpenApi Objects

5 participants